-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
BIP-458: Enable GOLD gauges on Arbitrum #571
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I pushed over this, can see comments in my last push. Create functions for pools and gauges are done outside of payloads because anyone can do them. The only portion you needed was correct, the grant of permissions, setRelativeWeightCap, and revoke. A second json was needed to add new gauges to the controller, via the LM multisig using gaugeAdderV4. We can discuss Friday or this weekend when you have time.
BIP validation results for file BIPs/2023-W42/BIP-458B.json:
BIP validation results for file BIPs/2023-W43/BIP-458A.json:
|
File name: BIPs/2023-W42/BIP-458B.json
File name: BIPs/2023-W43/BIP-458A.json
File name: BIPs/2023-W43/BIP-458A.json
|
Checked: https://arbiscan.io/address/0xf7c5c26B574063e7b098ed74fAd6779e65E3F836#readContract is a CL rate provider pointed at the cl wstETH/ETH exchange feed. |
- gauge adds missing (needs gauge adder v4) - this payload still needs to be modified
458A - DAO Multisig -reduced to only include the granting of setRelativeWeightCap role, setting Gold pool on Base to 5% cap, and revoking of role. Had to remove create functions, those are permisionless and not needed. Simulation: https://dashboard.tenderly.co/public/safe/safe-apps/simulator/0a864b21-f3b3-4a5a-bff1-6e4c430ef21e 458B - Uses the Maxi LM Multisig to add new gauges created to meet the cap requests. Simulation: https://dashboard.tenderly.co/public/safe/safe-apps/simulator/0a774ec0-f96f-4b6e-be87-0696d8d00150
4c251b0
to
a83797d
Compare
Reworked the branch, recommited the move of the new branch and force-pushed to clean up touching file from another BIP. |
BIP validation results for file BIPs/2023-W42/BIP-458B.json:
BIP validation results for file BIPs/2023-W43/BIP-458A.json:
|
File name: BIPs/2023-W42/BIP-458B.json
File name: BIPs/2023-W43/BIP-458A.json
File name: BIPs/2023-W43/BIP-458A.json
|
payload for proposed bip in https://forum.balancer.fi/t/bip-xxx-enable-50gold-25usdc-25wsteth-25gold-25bal-25aura-25wsteth-arbitrum/5262
this does not include the 8%/2% split for the ARB rewards, which is not something that is multisig controlled as i understand